Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add identity_id index to identity_verifiable_addresses table #2147

Merged
merged 1 commit into from
Jan 14, 2022

Conversation

zepatrik
Copy link
Member

Related issue(s)

The verifiable addresses are loaded eagerly into the identity. When that happens, the identity_verifiable_addresses table is queried by nid and identity_id. This index should greatly improve performance, especially of the /sessions/whoami endpoint.

@zepatrik zepatrik requested a review from aeneasr January 14, 2022 13:42
@codecov
Copy link

codecov bot commented Jan 14, 2022

Codecov Report

Merging #2147 (e29faaa) into master (78b652a) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2147      +/-   ##
==========================================
- Coverage   75.24%   75.21%   -0.03%     
==========================================
  Files         294      294              
  Lines       15614    15614              
==========================================
- Hits        11748    11744       -4     
- Misses       3032     3035       +3     
- Partials      834      835       +1     
Impacted Files Coverage Δ
session/test/persistence.go 98.61% <0.00%> (-1.39%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78b652a...e29faaa. Read the comment docs.

@aeneasr aeneasr merged commit 86fd942 into master Jan 14, 2022
@aeneasr aeneasr deleted the fix/verified-email-index branch January 14, 2022 14:44
@vinckr vinckr mentioned this pull request Mar 18, 2022
peturgeorgievv pushed a commit to senteca/kratos-fork that referenced this pull request Jun 30, 2023
…ory#2147)

The verifiable addresses are loaded eagerly into the identity. When that happens, the `identity_verifiable_addresses` table is queried by `nid` and `identity_id`. This index should greatly improve performance, especially of the `/sessions/whoami` endpoint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants